Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Port dspace-7_x] Communities & Collections tree browser updates - Replaced #2597 #2623

Merged
merged 2 commits into from
Nov 10, 2023

Conversation

dspace-bot
Copy link
Contributor

Port of #2611 by @wwtamu to dspace-7_x.

`loadingNode` ends up being the current `node` after clicking it preventing it from rendering when more pages available.

Update community list component spec

Make the show more flat node id unique

The nodes with same id are conflicting when added to the tree. Clicking on the second with same id places the show more button under the wrong branch and expands the wrong page.

(cherry picked from commit 11d3771)
Co-Authored-By: Art Lowel <[email protected]>
(cherry picked from commit dc2ef98)
@dspace-bot dspace-bot added 1 APPROVAL pull request only requires a single approval to merge component: Community Community display or editing component: Discovery related to discovery search or browse system usability labels Nov 10, 2023
@tdonohue tdonohue added this to the 7.6.1 milestone Nov 10, 2023
@tdonohue tdonohue merged commit d166b5e into dspace-7_x Nov 10, 2023
10 checks passed
@tdonohue tdonohue deleted the backport-2611-to-dspace-7_x branch November 10, 2023 15:55
4science-it pushed a commit to 4Science/dspace-angular that referenced this pull request Dec 13, 2024
[DSC-1899] Updated bitbucket-pipelines.yml with prod

Approved-by: Vincenzo Mecca
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 APPROVAL pull request only requires a single approval to merge component: Community Community display or editing component: Discovery related to discovery search or browse system usability
Projects
Development

Successfully merging this pull request may close these issues.

3 participants